Solid v2 beta 10#7284
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
View your CI Pipeline Execution ↗ for commit ad6acb6
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview6 package(s) bumped directly, 0 bumped as dependents. 🟩 Patch bumps
|
Bundle Size Benchmarks
Trend sparkline is historical gzip bytes ending with this PR measurement; lower is better. |
Merging this PR will degrade performance by 9.33%
Performance Changes
Comparing Footnotes |
This reverts commit 5b40496.
There was a problem hiding this comment.
Important
At least one additional CI pipeline execution has run since the conclusion below was written and it may no longer be applicable.
Nx Cloud is proposing a fix for your failed CI:
We re-apply the createRenderEffect/flatten shim in StartClient.tsx that was introduced in commit 5b4049675b and then inadvertently reverted, restoring compatibility with Solid.js v2 beta.9's updated reactive scheduler. Without this call, the RouterProvider JSX is not properly connected to Solid.js v2's reactive graph during client hydration, causing all client-rendered routes to produce blank pages and failing 22/29 server-function E2E tests and 1/6 basic-solid-query E2E tests.
Tip
✅ We verified this fix by re-running tanstack-solid-start-e2e-csp:test:e2e, tanstack-solid-start-e2e-css-modules:test:e2e, tanstack-solid-start-e2e-server-routes:test:e2e and 5 more.
diff --git a/packages/solid-start-client/src/StartClient.tsx b/packages/solid-start-client/src/StartClient.tsx
index 88fd53e732..4bfd8ecc51 100644
--- a/packages/solid-start-client/src/StartClient.tsx
+++ b/packages/solid-start-client/src/StartClient.tsx
@@ -1,6 +1,12 @@
+import * as Solid from 'solid-js'
import { RouterProvider } from '@tanstack/solid-router'
import type { AnyRouter } from '@tanstack/router-core'
export function StartClient(props: { router: AnyRouter }) {
- return <RouterProvider router={props.router} />
+ const routerProvider = <RouterProvider router={props.router} />
+ Solid.createRenderEffect(
+ () => Solid.flatten(routerProvider),
+ () => {},
+ )
+ return routerProvider
}
Or Apply changes locally with:
npx nx-cloud apply-locally GjBI-xyFa
Apply fix locally with your editor ↗ View interactive diff ↗
🎓 Learn more about Self-Healing CI on nx.dev
No description provided.